-
Notifications
You must be signed in to change notification settings - Fork 76
overridable: Release notes (breaking change, deprecation) + dynamicParametrize usage docs
#864
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
overridable: Release notes (breaking change, deprecation) + dynamicParametrize usage docs
#864
Conversation
…rize` usage for both built-in and custom fields
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The added information is great, and I have a suggestion for refactoring the organization of the whole document hahaha! Feel free to build on that / tweak to liking.
* Rearranged the content of `override_components.md` according to suggestions from @fenekku
ce2190b to
f5430c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes! I have 2 remaining comments, but it's good by me once you've addressed them.
Closes #863
Description
This PR documents the breaking changes introduced in inveniosoftware/invenio-rdm-records#2101 and the prop name deprecations (non-breaking) in inveniosoftware/react-invenio-forms#298 in the
vNextrelease/upgrade notes.Additionally, it adds documentation to the "How to override UI React components" and "UI widgets" pages explaining how to use
dynamicParametrizeto enable field prop overrides that respond to the form state. This feature was added in inveniosoftware/react-invenio-forms#300.It also documents the common props added to built-in deposit form field components also added in inveniosoftware/invenio-rdm-records#2101